Skip to content

[DOCS] Update list of Apache and Nginx ML modules#1681

Merged
lcawl merged 10 commits intoelastic:masterfrom
lcawl:legacy-modules
Jun 9, 2021
Merged

[DOCS] Update list of Apache and Nginx ML modules#1681
lcawl merged 10 commits intoelastic:masterfrom
lcawl:legacy-modules

Conversation

@lcawl lcawl changed the title [DOCS] Indicate Apache and Nginx ML modules are legacy versions [DOCS] Update list of Apache ML modules Jun 3, 2021
@lcawl lcawl added the v7.13.2 label Jun 3, 2021
@lcawl lcawl marked this pull request as ready for review June 3, 2021 21:35
@lcawl lcawl requested review from peteharverson and szabosteve June 3, 2021 21:35
@lcawl lcawl changed the title [DOCS] Update list of Apache ML modules [DOCS] Update list of Apache and Nginx ML modules Jun 3, 2021
Copy link
Copy Markdown
Contributor

@szabosteve szabosteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for updating the list! I have two minor comments. Otherwise, LGTM!

Comment thread docs/en/stack/ml/anomaly-detection/ootb-ml-jobs-nginx.asciidoc Outdated
Comment thread docs/en/stack/ml/anomaly-detection/ootb-ml-jobs-nginx.asciidoc
lcawl and others added 3 commits June 4, 2021 08:18
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Comment thread docs/en/stack/ml/anomaly-detection/ootb-ml-jobs-apache.asciidoc Outdated
Copy link
Copy Markdown

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The Nginx text needs a couple of edits.

Comment thread docs/en/stack/ml/anomaly-detection/ootb-ml-jobs-nginx.asciidoc Outdated
Comment thread docs/en/stack/ml/anomaly-detection/ootb-ml-jobs-nginx.asciidoc Outdated
Copy link
Copy Markdown

@peteharverson peteharverson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Latest edits LGTM

@lcawl lcawl merged commit 9c12937 into elastic:master Jun 9, 2021
@lcawl lcawl deleted the legacy-modules branch June 9, 2021 19:23
lcawl added a commit to lcawl/stack-docs that referenced this pull request Jun 9, 2021
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
lcawl added a commit to lcawl/stack-docs that referenced this pull request Jun 9, 2021
Co-authored-by: István Zoltán Szabó <istvan.szabo@elastic.co>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants